Skip to content

Update list of authorized CI users - #3241

Merged
timmoon10 merged 1 commit into
NVIDIA:mainfrom
timmoon10:tmoon/te_ci_users
Jul 22, 2026
Merged

Update list of authorized CI users#3241
timmoon10 merged 1 commit into
NVIDIA:mainfrom
timmoon10:tmoon/te_ci_users

Conversation

@timmoon10

Copy link
Copy Markdown
Member

Signed-off-by: Tim Moon <tmoon@nvidia.com>
@timmoon10
timmoon10 requested review from phu0ngng and ptrendx July 22, 2026 22:17
@timmoon10 timmoon10 added the testing Improvements to tests or testing infrastructure label Jul 22, 2026
@greptile-apps

greptile-apps Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR adds a single GitHub username (YangFei1990) to the allowlist of users authorized to trigger CI by commenting /te-ci on a pull request.

  • The change is confined to the Authorization job's if condition in .github/workflows/trigger-ci.yml, appending one || github.actor == 'YangFei1990' clause to the existing list of ~40 approved users.
  • No workflow logic, secrets handling, or other job steps are modified.

Confidence Score: 5/5

Safe to merge — the only change is appending one username to an existing CI allowlist, with no modifications to workflow logic or secrets handling.

A single || github.actor == 'YangFei1990' clause is added to an established allowlist of ~40 users. No job steps, secrets, conditions, or other behavior is affected; the risk surface is limited to granting one additional person the ability to trigger CI runs.

No files require special attention.

Important Files Changed

Filename Overview
.github/workflows/trigger-ci.yml Adds 'YangFei1990' to the authorized CI user allowlist; no logic changes.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["/te-ci comment posted on PR"] --> B{Comment starts with '/te-ci'?}
    B -- No --> Z[Workflow skipped]
    B -- Yes --> C{github.actor in allowlist?}
    C -- No --> Z
    C -- Yes --> D[Authorization job\nblossom-ci AUTH]
    D --> E[Vulnerability-scan job\nNVIDIA/blossom-action]
    E --> F[Job-trigger\nblossom-ci START-CI-JOB]
Loading

Reviews (1): Last reviewed commit: "Update list of authorized CI users" | Re-trigger Greptile

@timmoon10
timmoon10 merged commit 8abfa2e into NVIDIA:main Jul 22, 2026
10 of 14 checks passed
@timmoon10
timmoon10 deleted the tmoon/te_ci_users branch July 22, 2026 22:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testing Improvements to tests or testing infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants